Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix pointer problem in UnmanagedVector #571

Merged
merged 2 commits into from
Nov 13, 2024
Merged

Fix pointer problem in UnmanagedVector #571

merged 2 commits into from
Nov 13, 2024

Conversation

chipshort
Copy link
Collaborator

Fixes the problem reported in CosmWasm/wasmd#2011

Copy link
Contributor

@kulikthebird kulikthebird left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good change!

@kulikthebird
Copy link
Contributor

@Mergifyio backport release/1.5 release/2.0 release/2.1

Copy link

mergify bot commented Nov 13, 2024

backport release/1.5 release/2.0 release/2.1

✅ Backports have been created

@chipshort chipshort merged commit 7ba94cd into main Nov 13, 2024
14 checks passed
@chipshort chipshort deleted the fix-ptr-bug branch November 13, 2024 15:23
kulikthebird added a commit that referenced this pull request Nov 14, 2024
kulikthebird added a commit that referenced this pull request Nov 14, 2024
kulikthebird added a commit that referenced this pull request Nov 14, 2024
@webmaster128 webmaster128 changed the title Fix pointer problem Fix pointer problem in UnmanagedVector Dec 5, 2024
MikkySnow added a commit to SigmaGmbH/swisstronik-chain that referenced this pull request Feb 3, 2025
MikkySnow added a commit to SigmaGmbH/swisstronik-chain that referenced this pull request Feb 3, 2025
MikkySnow added a commit to SigmaGmbH/swisstronik-chain that referenced this pull request Feb 10, 2025
* feat: add migrations for v1.0.7 upgrade

* feat: add script for simulation of chain upgrade with 1 validator

* feat: add cli command to obtain verification holder by verification id

* fix: fix issue with upgrade handler

* feat: add hack to deploy Arachnid Proxy

* refactor: more meaningful error

* feat: add deployment of Arachnid Deterministic Deployer

* feat: add is_revoked to QueryVerificationsDetailsResponse

* feat: return only non-revoked verification details in x/compliance precompile

* refactor: remove redundant line

* refactor: return in precompile only verifications with active issuers

* refactor: prohibit to convert credential with inactive issuer

* feat: add ffi query to revoke verification

* feat: add precompile handler to revoke verification

* feat: update compliance proxy contract

* test: add test for revocation by issuer

* feat: provide caller address during credential revocation

* feat: add check for caller during credential revocation

* fix: fix issue with invalid non-revocation proof

* feat: add VT_BIOMETRIC verification type

* refactor: remove unused dependencies

* feat: add convertCredential function to precompile interface

* test: add test for converting credential

* feat: add convert credential function signature

* feat: add method to encode convert credential request

* feat: add precompile handler to convert credential

* feat: add cli command to get attached holder public key

* chore: intermediate commit

* feat: add convertCredential method to precompile

* test: fix test for converting v1 verification to v2

* fix: applying fix for pointer problem from CosmWasm/wasmvm#571

* test: add test for non-revocation proof

* refactor: remove unused consts and imports

* refactor: add index variable

* fix: fix build using `make build-docker-local`

* ci: use actions/upload-artifact@v4

* fix: add checks in keeper for max sizes of verification data fields

* test: add test for x/compliance keeper for case with huge original data

* fix: add checks for verification details sizes in compliance_bridge precompile

* feat: start refactoring self attestation to DCAP instead of deprecated EPID

* feat: add DCAP-based self attestation for mainnet / testnet checks

* chore: fix some clippy warnings

* chore: run cargo clippy --fix
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants